Tearing down https://woodpecker-ci-woodpecker-pr-4132.surge.sh

I would suggest to simply hard-code it in the cli to the same value we use in the agent. wp_%s_%d.

I would suggest to simply hard-code it in the cli to the same value we use in the agent. wp_%s_%d.

no hardcode will make colisins as we have right now ... but I could change the format to have a wp_ prefix. the problem is that we dont have a number that is even near the truth for each workflow

no hardcode will make colisins as we have right now ... but I could change the format to have a wp_ prefix. the problem is that we dont have a number that is even near the truth for each workflow

ulid.Make().String() is fine. I meant we don't need a cli option for it at all as users can't change it for the agent as well.

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 26.18%. Comparing base (b59533a) to head (c2be418).

@@            Coverage Diff             @@
##             main    #4132      +/-   ##
==========================================
- Coverage   26.20%   26.18%   -0.02%     
==========================================
  Files         373      373              
  Lines       27034    27034              
==========================================
- Hits         7084     7079       -5     
- Misses      19294    19299       +5     
  Partials      656      656              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

well you normally dont need it but why remove it ... if we should not present this to the user we can just hide that flag

*it's alrady hidden so why remove it etc ?!?

*it's alrady hidden so why remove it etc ?!?

Why do we need to support changing it? It's just extra code ...

Well for those who wana change it at some point could also just compile the cli anyway ...

@anbraten done